Conversation
tgraham-ucsc
left a comment
tgraham-ucsc
left a comment
There was a problem hiding this comment.
Hello Jaggi - I went through the work so far, nice work, it looks as if the training is going well and you're making good progress.
There was a problem hiding this comment.
Looks good, well formatted, proper use of the ref function. Nice work!
There was a problem hiding this comment.
This looks correct, we want the the project_name to be relevant to the subject area, and we want to build the staging layer as views and the marts layer as tables. Nice work.
There was a problem hiding this comment.
This model could be removed from the project as it's really just a placeholder for the project.
There was a problem hiding this comment.
This model could be removed from the project as it's really just a placeholder for the project.
There was a problem hiding this comment.
This file can likely be removed as dbt will handle the table creation pieces
There was a problem hiding this comment.
Looks good, nice use of the source function. We could probably remove the commented block if this were not just training.
There was a problem hiding this comment.
Looks good, nice use of the source function. We could probably remove the commented block if this were not just training.
There was a problem hiding this comment.
This looks good, nice use of ref and I like how it pulls all of the components together for a final model.
There was a problem hiding this comment.
Good to see the enhancements in the code!
2 participants
Jaggi 02/28/2024: Created my own model, models/marts/finance/fct_orders_jk1.sql. As an experiment, in my fct_orders_jk1.sql model, I multiplied the amount by 100. In the original models/staging/stripe/stg_stripe__payments.sql model, the amount is divided by 100.